Call really_destroy! in validate_roles, to not throw error when role … #1444
Conversation
| it "is valid with membership in different section active since today" do | ||
| create_role(:matterhorn_mitglieder, "Mitglied", start_on: Time.zone.today) | ||
| expect(switch).to be_valid | ||
| expect(switch).to be_valid |
There was a problem hiding this comment.
For some reason calling .delete only fails when calling this method twice? In our request this method is called twice, resulting in this bug, to recreate it in this spec we have to call it twice...
njaeggi
force-pushed
the
bug/1357/switch-stammsektion-with-new-role
branch
2 times, most recently
from
f6020fc to
a22fd73
Compare
There was a problem hiding this comment.
Dass der Bug nur auftritt wenn wir zweimal valid? aufrufen habe ich noch weitergeforscht.
Faktisch ist das Problem dass er beim zweiten mal aus irgendeinem Grund die old_role = roles_to_destroy.first welche als gelöscht gelten sollte doch noch in der overlap_validation findet. Beim ersten Mal klappts problemlos. Wir konnten das Phänomen soweit zurückfolgen dass in der DB aufgrund der Transaktion korrekt nichts geschrieben wird. Da das Ruby Objekt der old_role aber memoized ist, gibt es beim ersten valid? Call also wahrscheinlich changes auf dieses Objekt welches dann den zweiten valid? Call verwirrt?
TheWalkingLeek
force-pushed
the
bug/1357/switch-stammsektion-with-new-role
branch
from
dcc80ea to
9f6d72b
Compare
…will be deleted anyways